Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: correct Knip configuration for conversation-rewards #11

Conversation

gitcoindev
Copy link
Contributor

Resolves #10

@gitcoindev
Copy link
Contributor Author

QA: https://github.com/gitcoindev/conversation-rewards/actions/runs/8643695354/job/23697259210 and this pull request.

I fixed the workflow, dependencies and configuration issues. Knip is failing with a few real findings that should be addressed in other pull requests.

@gentlementlegen
Copy link
Member

gentlementlegen commented Apr 11, 2024

@gitcoindev Thanks for your help. Maybe Knip errors can be fixed through #8 directly. Could you also fix Jest at the same? It seems that ts-node is missing in the devDependencies.

@gitcoindev
Copy link
Contributor Author

@gentlementlegen I upgraded Knip to latest version, fixed all Knip warnings and Jest.

@molecula451 molecula451 merged commit 536a637 into ubiquity-os-marketplace:development Apr 11, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Correct Knip configuration for conversation-rewards
3 participants